Skip to content

STM32L432KC: increase RAM size from 48k to 64k #5171

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 27, 2017

Conversation

jeromecoutant
Copy link
Collaborator

Description

RAM size is increased from 48k to 64k for STM32L432KC as proposed in #4823

WARNING: you have to update ST-Link FW to V2J29M18
http://www.st.com/en/development-tools/stsw-link007.html

Status

READY

Tests

All OS2 and OS5 tests have been checked

Copy link
Contributor

@0xc0170 0xc0170 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What was the limitation before to use only 48k ? firmware?

@jeromecoutant
Copy link
Collaborator Author

What was the limitation before to use only 48k ? firmware?

2 things:

  1. yes, FW: only V2J29M18 FW allows to use more than 48k

  2. As explained in NUCLEO_L432KC memory definition enhancements should give 16kB (25%) more SRAM #4823, SRAM1 and SRAM2 are not in the same area. But SRAM2 is mapped continuously to SRAM1. We didn't know that...

Thx @helmut64

@helmut64
Copy link
Contributor

Changes look good, thank you enhancing this. Helmut

@jeromecoutant
Copy link
Collaborator Author

morph test

@0xc0170
Copy link
Contributor

0xc0170 commented Sep 27, 2017

/morph test

@mbed-bot
Copy link

Result: SUCCESS

Your command has finished executing! Here's what you wrote!

/morph test

Output

mbed Build Number: 1402

All builds and test passed!

@helmut64
Copy link
Contributor

helmut64 commented Oct 1, 2017

@jeromecoutant
I verified the changes, it works great for me.
00:10:47.080943 MyHome: 1.0 (Oct 1 2017 16:36:04 mbed: v151)
00:10:47.081408 InitDefaults Done
00:10:47.083461 MemoryAvailable: 55 kB (56720 bytes)
Thank you, Helmut

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants